Add mobile quick-add floating button and modal for quick transactions#2
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 53ecb6f68e
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| } else if (currentFilter === 'history') { | ||
| await renderHistoryView(); |
There was a problem hiding this comment.
Refresh the account view after quick-add saves
When the FAB is used from the Account view (currentFilter === 'account'), the new entry is added to expense[] but this refresh helper never calls renderAccountView(). The account cards are computed only inside renderAccountView() from the in-memory transaction list, so after adding an income/expense from that visible view the transaction count, income, expense, and balance remain stale until the user navigates away and back.
Useful? React with 👍 / 👎.
Motivation
Description
index.htmlthat appears on screens <768px (#mobileQuickAddBtn,#mobileQuickAddModal).script.js: new DOM refs,mobileQuickDatePicker(Flatpickr),addMobileQuickEntry,updateMobileQuickCategoryOptions,openMobileQuickAdd,closeMobileQuickAdd, andresetMobileQuickAddForm.saveTransactionandrefreshAfterTransactionAddedand updated the dashboard form to use them so both forms save consistently.Testing
node --check script.jsran and returned no syntax errors (success).git diff --checkwas run to validate whitespace/patch issues (success).npm cifailed due to registry permission errors (403 Forbiddenfor a dependency), so node modules could not be restored in this environment (failed).npm run build:csscould not complete because the Tailwind CLI was unavailable in the environment (failed), so inline mobile-only CSS was added to avoid requiring a rebuild.Codex Task